Skip to content

drop access report tooltips #1477 #1523

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 17, 2025

Conversation

MontrealSergiy
Copy link
Contributor

No description provided.

@MontrealSergiy MontrealSergiy self-assigned this Jun 16, 2025
@MontrealSergiy MontrealSergiy added User Interface Admin Features or bugs related to administrative features labels Jun 16, 2025
@MontrealSergiy MontrealSergiy linked an issue Jun 16, 2025 that may be closed by this pull request
@natacha-beck
Copy link
Contributor

I tested this one, it works on the access reports page where empty tool_tips are removed.

But should we also fix the html_tool_tip in RichUiHelper in order to ignore the content div in case content is blank ?

@MontrealSergiy
Copy link
Contributor Author

MontrealSergiy commented Jul 16, 2025

Easy, see #1537 but since Pierre did not ask for it, would it be better to open a separate issue?

@prioux
Copy link
Member

prioux commented Jul 17, 2025

I don't understand where this comes from. The changes seem to remove information in the reports? Why?

@prioux
Copy link
Member

prioux commented Jul 17, 2025

Ah I found the issue now

@prioux prioux merged commit 25b6d4e into aces:master Jul 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Features or bugs related to administrative features User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

More adjustments to access reports
3 participants